Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Public Release for Fall24 meta-release #148

Merged
merged 31 commits into from
Sep 6, 2024
Merged

Public Release for Fall24 meta-release #148

merged 31 commits into from
Sep 6, 2024

Conversation

bigludo7
Copy link
Collaborator

What type of PR is this?

Add one of the following kinds:

  • subproject management

What this PR does / why we need it:

Related to issue #109

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

Missing link for Test Statement for sim-swap

Changelog input

 Public release for sim-swap v1.0.0 and sim-swap-subscriptions v0.1.0

Additional documentation

This section can be blank.

docs

Change version to v1.0.0
Fix links for test definition
Fix links for test definition
Prepare API Checklist for sim swap subscription
Update readme for r1.2
Making aligned with other readme
update changelog for r1.2
update changelog
fix duplicate swap
Copy link

github-actions bot commented Aug 23, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 2 0 0.02s
✅ OPENAPI spectral 2 0 3.26s
✅ REPOSITORY git_diff yes no 0.01s
✅ REPOSITORY secretlint yes no 0.75s
✅ YAML yamllint 2 0 0.48s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

Copy link
Collaborator

@hdamker hdamker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions, one mandatory change.

Please add @camaraproject/release-management_maintainers if ready for review by Release Management

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
code/API_definitions/sim-swap-subscriptions.yaml Outdated Show resolved Hide resolved
Co-authored-by: Herbert Damker <[email protected]>
bigludo7 and others added 11 commits August 23, 2024 16:54
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
@bigludo7
Copy link
Collaborator Author

Huge thanks @hdamker for the review.

Copy link
Collaborator

@jlurien jlurien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Also correct: Given the resource "/sim-swap-subscriptions/v0/subscriptions" -> Given the resource "/sim-swap-subscriptions/v0.1/subscriptions"


Get timestamp of last MSISDN <-> IMSI pairing change for the provided phone number.
#
# Get timestamp of last MSISDN <-> IMSI pairing change for the provided phone number.

Background: Common retrieveSimSwapDate setup
Given the resource "sim-swap/v0/retrieve-date"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It must be v1

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done :)


check if SIM swap has been performed during a past period
#
# check if SIM swap has been performed during a past period

Background: Common checkSimSwap setup
Given the resource "sim-swap/v0/check"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It must be v1

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

bigludo7 and others added 4 commits September 3, 2024 14:31
Change sim-swap/v0/check to sim-swap/v1/check
Change sim-swap/v0/retrieve-date to sim-swap/v1/retrieve-date
Chenge /sim-swap-subscriptions/v0/subscriptions to /sim-swap-subscriptions/v0.1/subscriptions
@bigludo7
Copy link
Collaborator Author

bigludo7 commented Sep 3, 2024

@jlurien Thanks for the review. Update done for the 3 files.

@jlurien
Copy link
Collaborator

jlurien commented Sep 4, 2024

Let's wait to decision on camaraproject/ReleaseManagement#89 to merge this

@tanjadegroot
Copy link

Please see camaraproject/ReleaseManagement#89 on how to update the Test result statement in the API readiness checklist for this Fall24 meta-release.

Taking care of the Test result statement accordingly to camaraproject/ReleaseManagement#89
@bigludo7
Copy link
Collaborator Author

bigludo7 commented Sep 5, 2024

@jlurien @tanjadegroot aligned for the Test Statement.

Copy link
Collaborator

@jlurien jlurien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Release PR approved (on behalf of Release Management)

PR can now be merged and release r1.2 created. Please update https://wiki.camaraproject.org/display/CAM/SimSwap+API+Release+Tracking with date for M4 and link to r1.2 when it is created. Thanks

@bigludo7
Copy link
Collaborator Author

bigludo7 commented Sep 6, 2024

Thanks @jlurien for the review :)
As soon as @fernandopradocabrillo and/or @hdamker (for Axel) approve I merge & update the confluence.

Copy link
Collaborator

@fernandopradocabrillo fernandopradocabrillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bigludo7 bigludo7 merged commit f98b75b into main Sep 6, 2024
2 checks passed
@hdamker hdamker deleted the bigludo7/r1.2 branch September 6, 2024 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants